Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/user notes #1635

Closed
wants to merge 10 commits into from
Closed

Feature/user notes #1635

wants to merge 10 commits into from

Conversation

engelmav
Copy link

Using the AR implementation of CI with oci8 requires one to remove the double quote in $_escape_char within oci8_driver.php. For a trivial feature this is nowhere near obvious and should be highlighted in the tutorial and/or in the CI database section of documentation.

re: http://codeigniter.com/forums/viewthread/172990/#1018229

@narfbg
Copy link
Contributor

narfbg commented Jul 23, 2012

It is not required to remove it - if that was the case, it wouldn't be there in the first place. The double quote IS the escape character in Oracle databases. I'll have to dig some more into that to see why it complains about it in some cases though.

@narfbg
Copy link
Contributor

narfbg commented Nov 17, 2012

Turns out, when you escape identifiers in Oracle you're also explicitly setting them to be case-sensitive, so you must take care of writing field and table names as they were declared on creation.

@narfbg narfbg closed this Nov 17, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants